Skip to content

Conversation

@philippjfr
Copy link
Member

Not entirely happy with this as the editor isn't aligned quite right but I played with all the options and couldn't find one that produced the correct result.

Fixes #7295

@codecov
Copy link

codecov bot commented Sep 19, 2024

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.37%. Comparing base (977e801) to head (d09c0cd).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7299      +/-   ##
==========================================
- Coverage   87.40%   87.37%   -0.04%     
==========================================
  Files         347      347              
  Lines       53618    53618              
==========================================
- Hits        46867    46850      -17     
- Misses       6751     6768      +17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@hoxbro
Copy link
Member

hoxbro commented Sep 19, 2024

Seeing this problem for one of my examples in the original issue:
image

@philippjfr
Copy link
Member Author

Yep, seeing that too. Not sure what to do about it.

@hoxbro
Copy link
Member

hoxbro commented Sep 19, 2024

What about making this setting an option for now?

Or maybe document the behavior, as it can be enabled by the configuration directly: pn.widgets.Tabulator(... configuration={"popupContainer": True})

@philippjfr
Copy link
Member Author

So I'm leaning towards merging this as is now. It turns out the weird offset issue occurs only for the last column and in the meantime this PR also fixes:

#8096

and the coloring of the popup.

@philippjfr
Copy link
Member Author

Okay, fixed the offset issue by inserting an empty last column.

@philippjfr philippjfr merged commit 80d1daa into main Sep 1, 2025
1 check was pending
@philippjfr philippjfr deleted the popup_container branch September 1, 2025 14:34
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tabulator list editor inside a template / jupyter lab will not stay fixed to the cell

3 participants